Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Percy: GraphQL Mock Server #1063

Merged
merged 70 commits into from
Nov 16, 2023
Merged

Conversation

daniel-heppner-ibigroup
Copy link
Contributor

@daniel-heppner-ibigroup daniel-heppner-ibigroup commented Nov 8, 2023

Description:
This PR adds a new express.js server that handles GraphQL requests separately from REST requests. REST is handled by the HAR library that we already had, while GraphQL requests are handled by a GraphQL handler so that we can mock individual types on the GraphQL query and make decisions based on the arguments to it.

| Before | After |
|--------|-------|
| image | image |

miles-grant-ibigroup and others added 30 commits October 19, 2023 12:00
@daniel-heppner-ibigroup daniel-heppner-ibigroup changed the title Graphql mock server Percy: GraphQL Mock Server Nov 8, 2023
@daniel-heppner-ibigroup
Copy link
Contributor Author

daniel-heppner-ibigroup commented Nov 14, 2023

I can't figure out why the results in CI differ from the local results. In the snapshots:

  • The trip viewer is missing features showing the board and alight locations.
  • The stop ID is missing in the itinerary view.

Neither of these seem to be a problem locally, only in CI.

Then also the mobile view tests just fail entirely and I do not know why that is.

@miles-grant-ibigroup
Copy link
Collaborator

Assigning back to you until the fixes we did together are in

@miles-grant-ibigroup
Copy link
Collaborator

What are you stuck on right now again?

@daniel-heppner-ibigroup daniel-heppner-ibigroup merged commit 13ffcb2 into nearby-view Nov 16, 2023
5 of 7 checks passed
@daniel-heppner-ibigroup daniel-heppner-ibigroup mentioned this pull request Nov 16, 2023
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants